Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove AUGUR_RECURSION_LIMIT #1080

Merged
merged 1 commit into from
Aug 11, 2023
Merged

Conversation

joverlee521
Copy link
Contributor

The AUGUR_RECURSION_LIMIT is set to 10,000 by default since Augur 22.0.0 and the workflow's minimum required Augur version is now 22.0.1¹

¹

- augur=22.0.1

Testing

What steps should be taken to test the changes you've proposed?
If you added or changed behavior in the codebase, did you update the tests, or do you need help with this?

Release checklist

If this pull request introduces backward incompatible changes, complete the following steps for a new release of the workflow:

  • Determine the version number for the new release by incrementing the most recent release (e.g., "v2" from "v1").
  • Update docs/src/reference/change_log.md in this pull request to document these changes and the new version number.
  • After merging, create a new GitHub release with the new version number as the tag and release title.

If this pull request introduces new features, complete the following steps:

  • Update docs/src/reference/change_log.md in this pull request to document these changes by the date they were added.

The AUGUR_RECURSION_LIMIT is set to 10,000 by default since Augur 22.0.0
and the workflow's minimum required Augur version is now 22.0.1¹

¹ https://github.com/nextstrain/ncov/blob/d34a0cf67890fc6b33e003247b90e900ac02c80c/workflow/envs/nextstrain.yaml#L7
@joverlee521 joverlee521 requested a review from a team August 9, 2023 23:16
@joverlee521 joverlee521 merged commit d17782f into master Aug 11, 2023
13 checks passed
@joverlee521 joverlee521 deleted the remove-augur-recursion-limit branch August 11, 2023 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants